• Create ticket/issue

Follow the guide to enable codebase context checks.

Objective: Ensure all identifiers clearly express their purpose and intent, making code
self-documenting

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Objective: Ensure comprehensive error handling that provides meaningful context and graceful
degradation

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Objective: To prevent the leakage of sensitive system information through error messages while
providing sufficient detail for internal debugging.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Objective: To ensure logs are useful for debugging and auditing without exposing sensitive
information like PII, PHI, or cardholder data.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent
vulnerabilities

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Objective: To create a detailed and reliable record of critical system actions for security analysis
and compliance.

Status:
No logging: The new logic adds early-return conditions without any audit logging of the decision or
context, which may hinder traceability of critical planning decisions.

case *plan.Expr_Col:
	// Array types don't support zonemapping
	if expr.Typ.Id == int32(types.T_array_float32) || expr.Typ.Id == int32(types.T_array_float64) {
		return false
	}
	return true

Learn more about managing compliance generic rules or creating your own custom rules

Generalize the array type check to be more robust and future-proof. Instead of
checking for specific array types, use a prefix check on the type's string
representation to disable zonemapping for all array types.

pkg/sql/plan/utils.go [1101-1106]

 	case *plan.Expr_Col:
 		// Array types don't support zonemapping
-		if expr.Typ.Id == int32(types.T_array_float32) || expr.Typ.Id == int32(types.T_array_float64) {
+		if strings.HasPrefix(types.T(expr.Typ.Id).String(), "T_array") {
 			return false
 		}
 		return true

• Apply / Chat